Tweak publishing setup#2
Conversation
| name: Setup CI | ||
| description: Setup CI |
There was a problem hiding this comment.
out of curiosity - why do we need description too? I can't find the docs for this description field
There was a problem hiding this comment.
This is a weird one I think. In VS Code, I have the github actions extension (official) installed, which gives nice intellisense and stuff. For some reason it marked the whole file red without the description, so I added it to please it.
There was a problem hiding this comment.
I think this is the docs: https://docs.github.com/en/actions/reference/workflows-and-actions/metadata-syntax#description
| node-version: | ||
| description: Node.js version | ||
| required: false | ||
| default: 24 |
There was a problem hiding this comment.
The idea with defaulting to 22 was that our CI should test against the oldest supported version. I assume you have decided it's an overkill here - that's fine with me, I'm just doublechecking
There was a problem hiding this comment.
Yeah I figured it's simpler to only test node 24
No description provided.